Skip to content

ci: review the rewrite branch with CodeRabbit, now greptile is retired - #1136

Open
chappie-daemon wants to merge 1 commit into
rewrite/mainfrom
coderabbit-rewrite-main
Open

chappie-daemon wants to merge 1 commit into
rewrite/mainfrom
coderabbit-rewrite-main

Conversation

@chappie-daemon

@chappie-daemon chappie-daemon commented Nov 14, 2026 •

Copy link
Copy Markdown
Collaborator

Why

The rewrite's independent reviewer was greptile, run by the factory's static stage. It cannot run any more: its CLI sits on a paid tier the OSS allowance does not cover, the stored key came back revoked, and the service refused a 1,454-line branch as too large to send. Both failures recorded as "provider unavailable", which at a glance reads like a clean round — worse than having no reviewer.

CodeRabbit is already installed on this repository, and its free tier covers public repositories. It reviewed a PR on main here within seconds of being opened. It did not review the rewrite's PRs, because auto review covers the default branch and every rewrite PR targets rewrite/main.

What this adds

.coderabbit.yaml, with the default branch untouched:

  • reviews.auto_review.base_branches: ["^rewrite/main$"] — the one line that gives the rewrite its reviewer back.
  • auto_pause_after_reviewed_commits: 20 (default 5) — a job with fix rounds pushes more than five commits before its review rounds start, and the default pauses exactly then.
  • profile: chill, summaries and walkthroughs on; sequence diagrams and the poem off, since neither has been read.

Notes

  • Nothing else in review policy changes; the repository's own gates remain the merge gate, and no CodeRabbit approval is counted as a human one.
  • sequence_diagrams and auto-summaries in the PR description stay on their defaults except where named above.

Summary by CodeRabbit

  • Chores
    • Updated automated review settings for a development branch. This change does not affect the app’s features or behaviour for end users.

greptile was the factory's static reviewer and its CLI is no longer usable: a
paid tier the OSS allowance does not cover, a key that came back revoked, and a
diff cap that rejected a 1,454-line branch. CodeRabbit is already installed on
this repository and free on public ones, but its auto review covers the default
branch only, and every rewrite pull request targets rewrite/main.

base_branches adds that branch by name. auto_pause_after_reviewed_commits rises
from its default of 5, which a job with fix rounds would exhaust before its
review rounds start.
@changeset-bot

changeset-bot Bot commented Nov 14, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: fc84c54

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@coderabbitai

coderabbitai Bot commented Nov 14, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The pull request adds CodeRabbit review settings for Canadian English, the chill profile, review output options, and automatic reviews on matching base branches.

Changes: CodeRabbit review configuration

Layer / File(s) Summary
Review settings and automatic review conditions
.coderabbit.yaml
The configuration enables summaries and review status, disables sequence diagrams and poems, and enables automatic reviews for base branches matching ^rewrite/main$. It sets automatic pausing to 20 reviewed commits.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Merge Risk: Minimal · up to fc84c

This adds automatic CodeRabbit reviews for pull requests targeting rewrite/main. It has no runtime effect and does not change the existing merge gates, so it is safe to merge.

Architecture risk: Low · up to fc84c

The changed surface does not map to a changed system, dependency edge, entrypoint, or external dependency.

Changed systems: None identified.

Architecture concerns
No architecture-level concerns identified.

Review details

Before / after behavior

  • observed — Modified behavior in .coderabbit.yaml: Adds CodeRabbit configuration with Canadian English, the chill profile, summaries and review status enabled, sequence diagrams and poems disabled, and automatic review enabled for base branches matching ^rewrite/main$; automatic pausing is set to 20 reviewed commits. The accompanying comments describe the replacement and its context.
Pre-merge checks: Passed 4

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title says the rewrite branch is now reviewed by CodeRabbit because Greptile is retired. The change adds .coderabbit.yaml, which enables CodeRabbit auto-reviews for pull requests targeting rewrite…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Title check

Explanation

The title says the rewrite branch is now reviewed by CodeRabbit because Greptile is retired. The change adds .coderabbit.yaml, which enables CodeRabbit auto-reviews for pull requests targeting rewrite/main. The title matches the main change. It also names the Greptile retirement, which is context from the objectives and not a file change, but it does not misstate the change.

Finishing Touches

Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Recent review info

Run configuration

Commits

Reviewing files that changed from the base of the PR and between 509b403 and fc84c54.

Files selected for processing: 1

  • .coderabbit.yaml

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 8 remain after this review.

@greptile-apps

greptile-apps Bot commented Nov 14, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The PR appears safe to merge.

Summary

Adds .coderabbit.yaml to enable automatic reviews for PRs targeting rewrite/main.

  • CodeRabbit now reviews pull requests targeting rewrite/main automatically.

Reviews (1) · Last reviewed commit: "ci: review the rewrite branch with CodeR..." · Reviewed by Greptile

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant